Skip to content

fix(mobile): prevent message text overlap - #3506

Open
techieasif wants to merge 1 commit into
block:mainfrom
techieasif:asif/fix-mobile-message-overlap
Open

fix(mobile): prevent message text overlap#3506
techieasif wants to merge 1 commit into
block:mainfrom
techieasif:asif/fix-mobile-message-overlap

Conversation

@techieasif

@techieasif techieasif commented Jul 29, 2026

Copy link
Copy Markdown

Summary

Mobile text messages could leave stale paint over the following message or thread summary after #3370 disabled clipping on every message Material to support gallery overflow.

This change restores Clip.antiAlias for ordinary channel and thread messages while retaining Clip.none only when MessageContent detects an actual trailing multi-image gallery. That keeps gallery cards flush with the trailing gutter without exposing text-only rows to recycled-list paint bleed. A widget regression assertion verifies normal text rows are clipped; the existing gallery assertion continues to verify that galleries are not.

Manual verification: rendered the real mobile channel widget tree at the reporter's 475 px width with two thread summaries and confirmed each message and summary occupies its own row. No follow-up work is currently deferred.

Related issue

Fixes #3491.

Duplicate search: no competing open issue or PR found; #3491 is the closest and originating report.

Testing

  • just ci — passed (full Rust, desktop, Tauri, web, and mobile gate; 883 mobile tests passed, 1 skipped)
  • flutter test test/features/channels/channel_detail_page_test.dart — 52 passed
  • flutter test test/features/channels/message_content_test.dart — 46 passed
  • deterministic 475 px mobile widget render used for the after image below
Before After
Message text and reply summaries overlap. Message text and reply summaries remain in separate rows.
Before: overlapping mobile message rows After: non-overlapping mobile message rows

@techieasif
techieasif marked this pull request as ready for review July 29, 2026 10:14
@techieasif
techieasif requested a review from a team as a code owner July 29, 2026 10:14
Signed-off-by: Mohd Asif <techieasif@gmail.com>
@techieasif
techieasif force-pushed the asif/fix-mobile-message-overlap branch from 65941e7 to 82764e8 Compare July 30, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI glitch

1 participant